Skip to content

Migration Guide > Functional Components の翻訳 #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Nov 1, 2020

Conversation

jay-es
Copy link
Member

@jay-es jay-es commented Oct 22, 2020

resolve #116

https://v3.vuejs.org/guide/migration/functional-components.html の翻訳です。
なるべく「一語一句翻訳」のガイドラインに沿って翻訳しております。

32行目について、冒頭の Using のうまい訳が思いつかず、軽い意訳になっております。
118行目のリンクはページの翻訳が終わってからタイトルを合わせる必要がありそうです。(取り急ぎ、ガイドのタイトルに合わせてあります)

ご確認よろしくお願いします。

@netlify
Copy link

netlify bot commented Oct 22, 2020

Deploy preview for vuejs-v3-ja-doc-preview ready!

Built with commit fc80c14

https://deploy-preview-142--vuejs-v3-ja-doc-preview.netlify.app

@potato4d potato4d self-requested a review October 26, 2020 03:34
Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

概ね良さそうな気がします。Using の取り扱いについてコメントしました 📝


Using the `<dynamic-heading>` component, which is responsible for rendering out the appropriate heading (i.e., `h1`, `h2`, `h3`, etc.), this could have been written as a single-file component in 2.x as:
適切な見出し(つまり `h1`, `h2`, `h3` など)のレンダリングを担当する `<dynamic-heading>` コンポーネントは、2.x では単一ファイルコンポーネントとして次のように記述できました:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントにあった部分ここですよね。
<dynamic-heading> コンポーネントを使用すると〜から初めて先に Using the <dynamic-heading> component を消化し、which is ~ を読点のあとに持っていくのが読みやすいかもしれないですね

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L92 が使うと〜なので、同じように使うと〜で統一するほうが良さそうだろうか

Copy link
Member Author

@jay-es jay-es Oct 26, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@potato4d レビューありがとうございます!

提案いただいた内容ですと、以下のように直すイメージでよろしいでしょうか?

<dynamic-heading> コンポーネントを使うと、適切な見出し(つまり h1, h2, h3 など)のレンダリングを担当するコンポーネントは 2.x では単一ファイルコンポーネントとして次のように記述できました:

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ですです!

Copy link
Member Author

@jay-es jay-es Oct 30, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@potato4d 修正しましたのでご確認よろしくお願いします:pray:

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!

Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@potato4d potato4d merged commit 1f1cd2e into vuejs-jp:lang-ja Nov 1, 2020
@jay-es jay-es deleted the functional-components branch November 1, 2020 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migration Guide > Functional Components の翻訳
2 participants